Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update installation.rst #17

Merged
merged 4 commits into from
Jun 22, 2024
Merged

Conversation

leocov-dev
Copy link

updated to reflect being able to install from PyPi.
removed some things that don't make sense anymore.

Comment on lines -107 to -109
A specific branch named `no_big_files` is available for practical inclusion as a git submodule
in your own repository. This does not include the default database and example images, making
it less than 1 MB. For a specific application you probably want a custom database anyway.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this branch isn't valid in the cedar-solve fork and we do include the default database in the python wheel. the download size is only 8.1MB for the wheel.

Comment on lines 21 to 23
You can test that it works by invoking the database generator command::

You can test that it works by running the example provided in the GitHub repo.
tetra3-gen-db --help
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can't run the example when installing from a wheel, we don't include that directory or the images inside the wheel.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not? The idea is that the pip install is the minimum needed to run cedar-solve?

So if folks want to run the tests they can do the source code install documented in the installation.rst, right?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if they install it from pypi its assumed they are integrating it into their project in some way.
they can download the source or clone the git repo to run the tests and the example.

an alternative to including the example would be a minimal example described in the documentation as a tutorial.

A specific branch named `no_big_files` is available for practical inclusion as a git submodule
in your own repository. This does not include the default database and example images, making
it less than 1 MB. For a specific application you probably want a custom database anyway.

If problems arise
-----------------
Please get in touch by `filing an issue <https://github.com/smroid/cedar-solve/issues>`_.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While you're here, can you also add something like:

You can also join the Cedar Discord server at https://discord.gg/xbDrUyXP.

@@ -3,7 +3,7 @@ Installation

Getting Python
--------------
Cedar-solve is written for Python 3.7 or later (and therefore runs on almost any platform) and should
Cedar-solve is written for Python 3.8 or later (and therefore runs on almost any platform) and should
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was changed due to math.comb not existing in 3.7, the pyproject.toml file specifies >=3.8

Copy link
Owner

@smroid smroid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small addition to consider.

directly in your Python project as a module and use, or see below for how to install it.
You have two options to manually download the source code.

1. Download the release of your choice by going to `GitHub Releasess <https://github.com/smroid/cedar-solve/releases>`_
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spelling error on Releasess

Copy link
Owner

@smroid smroid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo, looks good.

@smroid smroid merged commit cf912c8 into smroid:master Jun 22, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants